Remove section about URCapX in development#521
Merged
Conversation
The External Control URCapX is considered maintained rather than developed by now, so it's fair to remove that statement. The robot setup instructions clearly cover that for PolyScope X another URCapX is needed, so that should suffice.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #521 +/- ##
==========================================
- Coverage 78.87% 77.90% -0.97%
==========================================
Files 116 116
Lines 6612 6609 -3
Branches 2920 2920
==========================================
- Hits 5215 5149 -66
- Misses 1031 1090 +59
- Partials 366 370 +4 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
The old screenshots listed all fieldbusses enabled, as well. Though technically, this is only affecting the firewall, that could be confusing, since we have strict incompatibility with fieldbusses being enabled.
urrsk
approved these changes
Jun 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The External Control URCapX is considered maintained rather than developed by now, so it's fair to remove that statement. The robot setup instructions clearly cover that for PolyScope X another URCapX is needed, so that should suffice.
Note
Low Risk
Documentation-only edit with no runtime or API impact.
Overview
Updates the PolyScope X compatibility note in
polyscope_compatibility.rstby removing the bullet that external control on PolyScope X needs a separate External Control URCapX that was described as still being created, with a link to the URCapX repository.The note still lists Dashboard client limitations on PolyScope X; setup guidance for the URCapX remains in the robot setup docs.
Reviewed by Cursor Bugbot for commit 1a936f7. Bugbot is set up for automated code reviews on this repo. Configure here.